Skip to content

[Fix] 030-livekit-agents-python — Pin deepgram-sdk==6.1.1 and add required tag#132

Merged
lukeocodes merged 1 commit intomainfrom
fix/030-livekit-agents-python-regression-2026-04-02
Apr 2, 2026
Merged

[Fix] 030-livekit-agents-python — Pin deepgram-sdk==6.1.1 and add required tag#132
lukeocodes merged 1 commit intomainfrom
fix/030-livekit-agents-python-regression-2026-04-02

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot commented Apr 2, 2026

Fix: 030-livekit-agents-python

Root cause: Outdated SDK pin (deepgram-sdk>=3.0.0) and missing tag="deepgram-examples" on test API call.

Changes:

  • Pin deepgram-sdk==6.1.1 in requirements.txt (was >=3.0.0)
  • Add tag="deepgram-examples" to the transcribe_url() call in tests/test_example.py

Tests after fix

Tests exit with code 2 (missing credentials) — expected in CI without secrets configured. No code bugs found.


Fix by Lead on 2026-04-02

…nts-python

- Update deepgram-sdk from >=3.0.0 to ==6.1.1 to match repo standard
- Add tag="deepgram-examples" to test transcribe_url call
@github-actions github-actions bot added the type:fix Bug fix label Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor Author

github-actions bot commented Apr 2, 2026

Code Review

Overall: APPROVED

Tests ran ✅

============================= test session starts ==============================
platform linux -- Python 3.11.15, pytest-9.0.2, pluggy-1.6.0
collecting ... collected 2 items

tests/test_example.py::test_deepgram_stt PASSED                          [ 50%]
tests/test_example.py::test_agent_module_imports PASSED                  [100%]

======================== 2 passed, 2 warnings in 2.76s =========================

Both tests pass with real credentials. Deepgram STT returned a valid transcript (length > 10). Agent module imports cleanly.

Integration genuineness

Pass — src/agent.py imports livekit.agents, livekit.plugins.silero, and wires Deepgram via inference.STT("deepgram/nova-3"). .env.example lists five real platform credentials (LiveKit, Deepgram, OpenAI). Test exits with code 2 when credentials are missing.

Code quality

  • ✅ Official Deepgram SDK pinned to deepgram-sdk==6.1.1 (required version)
  • tag="deepgram-examples" present on transcribe_url() call in test
  • ✅ No hardcoded credentials — all sourced from environment
  • ✅ Credential check runs before SDK imports (lines 6–15 of test)
  • ✅ Transcript assertion uses len(transcript) > 10 (length-based, not word-list)
  • ✅ No raw HTTP to Deepgram — uses official SDK

Documentation

  • ✅ README covers what-you'll-build, all env vars with where-to-get links, install/run instructions
  • .env.example present and complete (5 variables)

✓ All checks pass. Ready for merge.


Review by Lead on 2026-04-02

@github-actions github-actions bot added the status:review-passed Self-review passed label Apr 2, 2026
@lukeocodes lukeocodes merged commit fd86fdd into main Apr 2, 2026
1 check passed
@lukeocodes lukeocodes deleted the fix/030-livekit-agents-python-regression-2026-04-02 branch April 2, 2026 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:review-passed Self-review passed type:fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant